Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore:added a logrus hook for sentry #201

Merged
merged 9 commits into from
Sep 12, 2023
Merged

chore:added a logrus hook for sentry #201

merged 9 commits into from
Sep 12, 2023

Conversation

CommanderStorm
Copy link
Member

@CommanderStorm CommanderStorm commented Aug 29, 2023 •
edited
Loading

Resolves #98

Tested via setting the DSN and causing this exception:

for {
    log.WithField("user-who-created-this-error", "Frank").WithTime(time.Now()).WithError(errors.New("custom, very elaborate error message")).Error("this is a fatal excception")
}

yields
image

@CommanderStorm CommanderStorm self-assigned this Aug 29, 2023
@CommanderStorm CommanderStorm changed the title chore:added logrus_sentry to centralise error reporting chore:added a logrus hook for sentry Aug 29, 2023
@CommanderStorm CommanderStorm merged commit d7965bc into main Sep 12, 2023
@CommanderStorm CommanderStorm deleted the logrus_sentry branch September 12, 2023 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Antonwy Antonwy Awaiting requested review from Antonwy

@joschahenningsen joschahenningsen Awaiting requested review from joschahenningsen

Assignees

@CommanderStorm CommanderStorm

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve logging & monitoring
1 participant
@CommanderStorm